81ac490983b19f077a2b34366d56842fa6176e42,src/gr/uom/java/xmi/diff/UMLModelDiff.java,UMLModelDiff,checkForOperationMoves,#,719
Before Change
// Methods in the same class with similar body but different signature
}
}
else if(isSubclassOf(removedOperation.getClassName(), addedOperation.getClassName()) && equalParameters(removedOperation, addedOperation)) {
refactoring = new PullUpOperationRefactoring(removedOperation, addedOperation);
}
else if(isSubclassOf(addedOperation.getClassName(), removedOperation.getClassName()) && equalParameters(removedOperation, addedOperation)) {
After Change
List<UMLOperationBodyMapper> firstMappers = operationBodyMapperMap.get(operationBodyMapperMap.lastKey());
removedOperationIterator.remove();
for(UMLOperationBodyMapper firstMapper : firstMappers) {
UMLOperation addedOperation = firstMapper.getOperation2();
//addedOperations.remove(addedOperation);
Refactoring refactoring = null;
if(removedOperation.getClassName().equals(addedOperation.getClassName())) {
if (equalParameters(removedOperation, addedOperation)) {
refactoring = new RenameOperationRefactoring(removedOperation, addedOperation);
} else {
// Methods in the same class with similar body but different signature
}
}
else if(isSubclassOf(removedOperation.getClassName(), addedOperation.getClassName()) && (equalParameters(removedOperation, addedOperation) || overloadedParameters(removedOperation, addedOperation))) {
refactoring = new PullUpOperationRefactoring(removedOperation, addedOperation);
}
else if(isSubclassOf(addedOperation.getClassName(), removedOperation.getClassName()) && (equalParameters(removedOperation, addedOperation) || overloadedParameters(removedOperation, addedOperation))) {